Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update batterienauswertung.js #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

dog-rgb
Copy link

@dog-rgb dog-rgb commented May 23, 2023

Für die LowBatt Auswertung: Nur Geräte berücksichtigen die nicht den Status warn oder dead haben

Für die LowBatt Auswertung: Nur Geräte berücksichtigen die nicht den Status warn oder dead haben
@Pittini
Copy link
Owner

Pittini commented May 23, 2023

Für die LowBatt Auswertung: Nur Geräte berücksichtigen die nicht den Status warn oder dead haben

Bin mir grad nicht sicher ob das ne gute Idee ist und was Du damit bezweckst. Wenn ich das auf die schnelle richtig gesehen habe, werden durch die Änderung Geräte welche als tot gelten nicht mehr reaktiviert wenn se sich doch wieder melden.

@dog-rgb
Copy link
Author

dog-rgb commented May 23, 2023

Aus meiner Sicht hat die Änderung nur mit der Anzeige des Gerätes mit dem geringsten Batteriestand zu tun.

Ich hatte folgende Situation beobachtet: Ein Gerät hat sich zuletzt mit 29% Batteriestand gemeldet, danach war die Batterie leer und das Gerät meldete sich nicht mehr. Daher wurde es als "dead" gekennzeichnet. Der Batteriestand wurde aber immer noch mit 29% angezeigt. Und da dies auch der geringste Wert war, wurde dieses auch als LowBatt gewertet (aber nicht angezeigt). Durch meine Änderung werden Geräte die als "dead" (oder "warn") gewertet werden, nicht für die LowBatt Anzeige berücksichtigt.
Ich habe nur bei der if-Abfrage in Zeile 386 einen Block eingefügt um die Zeilen 387-390 nur dann auszuführen wenn das Gerät kein "warn" oder "dead" Status hat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants